Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set IsTabStop=False on scroll buttons in TabView #7539

Merged
merged 1 commit into from
Jul 29, 2022

Conversation

kmahone
Copy link
Member

@kmahone kmahone commented Jul 29, 2022

The Scroll Left/Right buttons in TabView are removed from the UIA tree. But they are still keyboard focusable. This gives a very bad experience when using Narrator. If you tab to these buttons, Narrator has no info about them and remains silent. This is very confusing.

It is not necessary for these buttons to be keyboard focusable. You can navigate and scroll through the tab list using the left and right arrows, so there is no reason to invoke these scroll buttons when using keyboard navigation.

Ideally, in addition to removing these buttons from the tab order, we would also add these buttons back to the UIA tree. This would enable them to be invoked by tools such as Voice Access. However, these buttons are inside the ListView template, and ListView has custom logic for creating its UIA children - it only shows the listview items, so other general UI under the ListView template does not show up. Hence these buttons do not appear even if they are not in the 'Raw' tree. The tab strip is still navigable with Voice Access using the "press right" and "press left" commands, even if it is not ideal.

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Jul 29, 2022
@kmahone
Copy link
Member Author

kmahone commented Jul 29, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kmahone kmahone merged commit 93e23c2 into main Jul 29, 2022
@kmahone kmahone deleted the user/kmahone/tabscrollbtn-main branch July 29, 2022 21:30
@ghost
Copy link

ghost commented Aug 15, 2022

🎉Microsoft.UI.Xaml v2.8.1 has been released which incorporates this pull request.:tada:

Handy links:

@ranjeshj ranjeshj added area-TabView team-Controls Issue for the Controls team and removed needs-triage Issue needs to be triaged by the area owners labels Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-TabView team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants